Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[microNPU] Enforce bias when pattern matching conv2d #9244

Merged
merged 1 commit into from
Oct 12, 2021

Conversation

lhutton1
Copy link
Contributor

Currently a conv2d pattern is matched when no bias is present. However, legalization expects a bias to be present, therefore causing an error when this is not the case. For now, enforce a bias when offloading conv2d to the NPU.

cc @ekalda @mbaret @manupa-arm @dchauhan-arm

Currently a conv2d pattern is matched when no bias is present.
However, legalization expects a bias to be present, therefore
causing an error when this is not the case. For now, enforce a
bias when offloading conv2d to the NPU.

Change-Id: I7f74b0f2c151f51ddc66ee1c5ebb77534238909b
Copy link
Contributor

@manupak manupak left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

Copy link
Contributor

@mbaret mbaret left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@mbaret mbaret merged commit 0d10973 into apache:main Oct 12, 2021
masahi pushed a commit to Laurawly/tvm-1 that referenced this pull request Oct 14, 2021
Currently a conv2d pattern is matched when no bias is present.
However, legalization expects a bias to be present, therefore
causing an error when this is not the case. For now, enforce a
bias when offloading conv2d to the NPU.

Change-Id: I7f74b0f2c151f51ddc66ee1c5ebb77534238909b
@lhutton1 lhutton1 deleted the enforce-bias branch October 28, 2021 16:51
ylc pushed a commit to ylc/tvm that referenced this pull request Jan 7, 2022
Currently a conv2d pattern is matched when no bias is present.
However, legalization expects a bias to be present, therefore
causing an error when this is not the case. For now, enforce a
bias when offloading conv2d to the NPU.

Change-Id: I7f74b0f2c151f51ddc66ee1c5ebb77534238909b
ylc pushed a commit to ylc/tvm that referenced this pull request Jan 13, 2022
Currently a conv2d pattern is matched when no bias is present.
However, legalization expects a bias to be present, therefore
causing an error when this is not the case. For now, enforce a
bias when offloading conv2d to the NPU.

Change-Id: I7f74b0f2c151f51ddc66ee1c5ebb77534238909b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants